Skip to content

Conversation

@Iran-110
Copy link
Contributor

@Iran-110 Iran-110 commented Mar 10, 2025

πŸ”— Linked issue

resolves #1118
resolves #1427

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

It helps to handle runtime configs to override the options in practice.

@Iran-110 Iran-110 requested a review from danielroe as a code owner March 10, 2025 21:46
@codecov-commenter
Copy link

codecov-commenter commented Mar 10, 2025

Codecov Report

βœ… All modified and coverable lines are covered by tests.
βœ… Project coverage is 7.03%. Comparing base (6022fa8) to head (7cf1884).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1738   +/-   ##
=====================================
  Coverage   7.03%   7.03%           
=====================================
  Files         77      77           
  Lines       3567    3567           
  Branches     138     138           
=====================================
  Hits         251     251           
  Misses      3268    3268           
  Partials      48      48           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@danielroe
Copy link
Member

is there a reason why you need to use public? shouldn't NUXT_IPX_ALIAS work just as well, given that the endpoint is only called on the server side?

@majtanmarek
Copy link

is any chance to merge this?

@danielroe
Copy link
Member

@majtanmarek I believe you should already be able to configure IPX using runtime config. This is purely a documentation update.

@danielroe danielroe changed the title feat: injects public environment variables in runtime docs: explain how to configure ipx at runtime Oct 30, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 30, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@nuxt/image@1738

commit: 7cf1884

@danielroe danielroe merged commit 8d547e8 into nuxt:main Oct 30, 2025
8 of 9 checks passed
@github-actions github-actions bot mentioned this pull request Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request for Enhancing Nuxt Image Module with Environment Variable Support feat: allow to edit image config at runtime

4 participants